Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactoring: Separating HSM and Speed sends #3063

Merged
merged 34 commits into from
Jan 3, 2025
Merged

Conversation

akanshaaa19
Copy link
Member

No description provided.

@akanshaaa19 akanshaaa19 added the enhancement New feature or request label Sep 4, 2024
@akanshaaa19 akanshaaa19 self-assigned this Sep 4, 2024
Copy link

github-actions bot commented Sep 4, 2024

@github-actions github-actions bot temporarily deployed to pull request September 4, 2024 18:17 Inactive
Copy link

codecov bot commented Sep 4, 2024

Codecov Report

Attention: Patch coverage is 78.50467% with 138 lines in your changes missing coverage. Please review.

Project coverage is 81.87%. Comparing base (5b65e56) to head (78a5145).
Report is 8 commits behind head on master.

Files with missing lines Patch % Lines
src/containers/SpeedSend/SpeedSend.tsx 73.50% 33 Missing and 20 partials ⚠️
src/containers/HSM/HSM.tsx 81.15% 32 Missing and 17 partials ⚠️
src/containers/HSM/HSMList/HSMList.tsx 63.07% 18 Missing and 6 partials ⚠️
...ntainers/SpeedSend/SpeedSendList/SpeedSendList.tsx 78.94% 6 Missing and 2 partials ⚠️
src/containers/HSM/HSM.helper.ts 95.34% 0 Missing and 2 partials ⚠️
src/mocks/Template.tsx 93.10% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3063   +/-   ##
=======================================
  Coverage   81.86%   81.87%           
=======================================
  Files         326      327    +1     
  Lines       10506    10516   +10     
  Branches     2167     2170    +3     
=======================================
+ Hits         8601     8610    +9     
- Misses       1296     1297    +1     
  Partials      609      609           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

cypress bot commented Sep 4, 2024

Glific    Run #5488

Run Properties:  status check failed Failed #5488  •  git commit c939fbee8c ℹ️: Merge 78a51456e834a30eda8657b86bd3f2ec30cb0693 into f061508f72adf0ccb7567302b58a...
Project Glific
Branch Review templates-refactoring
Run status status check failed Failed #5488
Run duration 28m 10s
Commit git commit c939fbee8c ℹ️: Merge 78a51456e834a30eda8657b86bd3f2ec30cb0693 into f061508f72adf0ccb7567302b58a...
Committer Akansha Sakhre
View all properties for this run ↗︎

Test results
Tests that failed  Failures 1
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 171
View all changes introduced in this branch ↗︎

Tests for review

Failed  cypress/e2e/flow/FlowEditor.spec.ts • 1 failed test

View Output

Test Artifacts
Flow > should configure Flow Test Replay Screenshots

@github-actions github-actions bot temporarily deployed to pull request September 20, 2024 18:20 Inactive
@github-actions github-actions bot temporarily deployed to pull request September 21, 2024 04:23 Inactive
@github-actions github-actions bot temporarily deployed to pull request September 23, 2024 16:43 Inactive
@github-actions github-actions bot temporarily deployed to pull request September 24, 2024 17:34 Inactive
@github-actions github-actions bot temporarily deployed to pull request September 24, 2024 18:08 Inactive
@github-actions github-actions bot temporarily deployed to pull request September 24, 2024 18:20 Inactive
@github-actions github-actions bot temporarily deployed to pull request September 24, 2024 18:24 Inactive
@github-actions github-actions bot temporarily deployed to pull request September 24, 2024 18:27 Inactive
@github-actions github-actions bot temporarily deployed to pull request September 29, 2024 05:39 Inactive
@github-actions github-actions bot temporarily deployed to pull request September 29, 2024 06:24 Inactive
@akanshaaa19 akanshaaa19 requested a review from kurund October 16, 2024 08:02
@github-actions github-actions bot temporarily deployed to pull request October 16, 2024 15:39 Inactive
@github-actions github-actions bot temporarily deployed to pull request October 17, 2024 03:46 Inactive
Copy link
Contributor

@kurund kurund left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you resolve the conflicts whenever you have some time?

@github-actions github-actions bot temporarily deployed to pull request December 18, 2024 04:57 Inactive
@github-actions github-actions bot temporarily deployed to pull request December 18, 2024 07:21 Inactive
@akanshaaa19 akanshaaa19 requested a review from kurund December 18, 2024 10:33
@github-actions github-actions bot temporarily deployed to pull request January 2, 2025 16:32 Inactive
@github-actions github-actions bot temporarily deployed to pull request January 2, 2025 16:40 Inactive
@github-actions github-actions bot temporarily deployed to pull request January 2, 2025 16:43 Inactive
@github-actions github-actions bot temporarily deployed to pull request January 3, 2025 06:50 Inactive
@github-actions github-actions bot temporarily deployed to pull request January 3, 2025 10:54 Inactive
@github-actions github-actions bot temporarily deployed to pull request January 3, 2025 11:31 Inactive
Copy link
Contributor

@kurund kurund left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

I have noticed some issues with Speed Send which are not related to this clean up. I will file separate issue for it.

@kurund kurund merged commit 6a4524d into master Jan 3, 2025
9 of 13 checks passed
@kurund kurund deleted the templates-refactoring branch January 3, 2025 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants